Skip to content

Fix nullable content across all social publishers#5

Merged
paulocastellano merged 3 commits into
mainfrom
fix/nullable-content-publishers
Mar 29, 2026
Merged

Fix nullable content across all social publishers#5
paulocastellano merged 3 commits into
mainfrom
fix/nullable-content-publishers

Conversation

@paulocastellano
Copy link
Copy Markdown
Contributor

Summary

  • Handle nullable content in all social media publishers to prevent TypeError crashes in production
  • Add per-platform validation: YouTube requires title, Threads/Facebook text-only posts require content, X requires text or media
  • Change PublishToSocialPlatform job to catch \Throwable instead of \Exception
  • Remove unused Threads reply scopes
  • Add null content tests for Instagram, Threads, Facebook, X, and YouTube publishers

Test plan

  • Verify all 666 tests pass
  • Test publishing with empty caption on Instagram, Facebook, Threads
  • Test publishing on YouTube without content (should show error)
  • Test publishing on X with only media (should work)

@paulocastellano paulocastellano merged commit 557cd21 into main Mar 29, 2026
2 checks passed
@paulocastellano paulocastellano deleted the fix/nullable-content-publishers branch March 29, 2026 20:34
paulocastellano added a commit that referenced this pull request Apr 1, 2026
…eckSocialConnections includes TokenExpired
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant